-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enable building of spack issm installation with wrappers #186
base: main
Are you sure you want to change the base?
Conversation
Hi @justinh2002 , Can you please show what you tested in this PR? |
ran
|
Hi @justinh2002 , Can you please test a |
771db2e
to
1165414
Compare
Hi @justinh2002 , Can you please add this as the first
|
@@ -12,46 +14,112 @@ class Issm(AutotoolsPackage): | |||
"""Ice-sheet and Sea-Level System Model""" | |||
|
|||
homepage = "https://issm.jpl.nasa.gov/" | |||
git = "https://github.com/ISSMteam/ISSM.git" | |||
git = "https://github.com/ACCESS-NRI/ISSM.git" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Stick to the style used in the rest of the SPRs.
…o one directory and clears it after compilation
|
Does this need to be rebased to main before we put it in |
Hi @justinh2002 , This needs to be tested, verified against Gadi non-spack build with wrappers, reviewed and merged before it appears on vk83. |
# Copy the entire source tree into an additional directory | ||
install_tree(self.stage.source_path, prefix.src) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure that the ISSM Spack package should install all the source? That's very unusual.
Hi @justinh2002 , Make a new Pull Request for the |
see #184
enable building of Spack issm package installation with wrappers - this allows matlab/python dependencies to be installed; so that Spack can be used to compile issm with pre- and post- processing features, either on a local machine or remote cluster.